Skip to content

fix(no-array-prototype-extensions): handle Promise.withResolvers - #2837

Merged
NullVoxPopuli merged 1 commit into
ember-cli:masterfrom
Guflly:fix/promise-with-resolvers-reject
Aug 17, 2026
Merged

fix(no-array-prototype-extensions): handle Promise.withResolvers#2837
NullVoxPopuli merged 1 commit into
ember-cli:masterfrom
Guflly:fix/promise-with-resolvers-reject

Conversation

@Guflly

@Guflly Guflly commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Fixes #2360.

Recognizes values initialized by Promise.withResolvers() (including window.Promise) so reject() is not reported as an Ember Array extension. Includes regression cases for both forms.

Tests:

  • pnpm test:coverage
  • pnpm lint

@NullVoxPopuli

Copy link
Copy Markdown
Contributor

Does this test fail without the change?

@Guflly

Guflly commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

Yes — I verified both new cases against the base rule, and each fails with one no-array-prototype-extensions error while both pass with the change.

@NullVoxPopuli
NullVoxPopuli merged commit 2e6973b into ember-cli:master Aug 17, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

no-array-prototype-extensions false positive Promsie.withResolvers

2 participants